Skip to content

Conversation

@MythreyaK
Copy link
Contributor

Motivation

Technical Details

JIRA ID

Test Plan

Test Result

Submission Checklist

@github-actions github-actions bot added the github actions Pull requests that update GitHub Actions code label Jan 8, 2026
@MythreyaK MythreyaK force-pushed the users/mkuriche/rocprofiler-sdk-fmt-build-fix-memory-header branch from baf8a4c to c006722 Compare January 10, 2026 00:19
@jbonnell-amd jbonnell-amd marked this pull request as ready for review January 12, 2026 15:24
@jbonnell-amd jbonnell-amd requested review from a team as code owners January 12, 2026 15:24
@jbonnell-amd jbonnell-amd merged commit 36d9d33 into develop Jan 12, 2026
27 of 40 checks passed
@jbonnell-amd jbonnell-amd deleted the users/mkuriche/rocprofiler-sdk-fmt-build-fix-memory-header branch January 12, 2026 17:59
systems-assistant bot pushed a commit to ROCm/rocprofiler-sdk that referenced this pull request Jan 12, 2026
 (#2537)

* [rocprofiler-sdk] Fix fmt::join build errors

- remedy use of fmt::join without include <fmt/ranges.h>

* include memory header

* Disable FMT build for SDK CI

* Add -DROCPROFILER_BUILD_FMT=OFF to sanitizer steps

* Add temporary workaround for rccl.h issue

* Add ROCPROFILER_INTERNAL_RCCL_API_TRACE to SDK CI builds

* disable clang-tidy for vendored includes
[rocm-systems] ROCm/rocm-systems#2537 (commit 36d9d33)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github actions Pull requests that update GitHub Actions code organization: ROCm project: rocprofiler-sdk

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants